Use uv #41
Merged
Use uv #41
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
keeping flit_core as build backend for now, since uv_build seems to be missing some things, like dynamic version
so that uv recognizes the project
not the input dir
losing XTNW059
There was a problem hiding this comment.
Pull request overview
This PR migrates the project's dependency management and task execution from pip/flit/conda to uv, a modern Python package installer and resolver. The migration involves updating build configurations, CI workflows, documentation, and development instructions to use uv commands throughout.
- Migrates dependency management from pip/flit to uv with PEP 735 dependency groups
- Updates CI/CD workflows and ReadTheDocs configuration to use uv for installation and execution
- Adds development documentation with uv commands and reorganizes panel app deployment
- Removes outdated matplotlib imports and plotting code from data preparation scripts
- Corrects data by removing duplicate XTNW059 entry from D'Addario string dataset
Reviewed changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Updates Python version constraint from ~=3.10 to >=3.10,<4, moves dependency groups from optional-dependencies to PEP 735 [dependency-groups], and adds new panel dependency group |
| .github/workflows/ci.yml | Replaces setup-python with astral-sh/setup-uv, updates all commands to use uv sync and uv run, refactors data prep script testing, and adds panel app build artifact upload |
| .readthedocs.yaml | Migrates from pip-based installation to uv with asdf, adds pre-build step to generate panel app |
| README.md | Adds comprehensive development section with uv commands for setup, testing, building panel app, and docs |
| panel/README.md | Updates development commands to use uv run and removes conda environment setup instructions |
| docs/panel.rst | Changes iframe source from external CDN URL to local static file path |
| docs/conf.py | Adds html_static_path configuration needed for serving static panel app |
| docs/_static/panel/.gitignore | Creates gitignore to exclude generated panel app files while preserving directory structure |
| .gitignore | Adds uv.lock to gitignore |
| data/worth.py | Removes unused matplotlib imports and plotting code |
| data/aquila.py | Removes unused matplotlib imports and comments out debugging plots |
| stringcalc/data/daddario-stp.csv | Removes duplicate/incorrect XTNW059 string entry |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
instead of `uv add`
otherwise uv downloads (except for 3.12)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.